Skip to content

Add {findings} template variable to hook commands#187

Merged
wesm merged 1 commit intoroborev-dev:mainfrom
jzila:add-findings-template-var
Feb 3, 2026
Merged

Add {findings} template variable to hook commands#187
wesm merged 1 commit intoroborev-dev:mainfrom
jzila:add-findings-template-var

Conversation

@jzila
Copy link
Contributor

@jzila jzila commented Feb 1, 2026

Summary

  • Adds a {findings} template variable to hook command interpolation, containing the full review output from the agent
  • The variable is shell-escaped like all other template variables to prevent injection
  • Populated on review.completed events

Test plan

  • Existing interpolation tests updated and passing
  • Shell injection tests extended to cover {findings}
  • Full test suite passes (go test ./...)

Allow hook commands to access the full review output via {findings},
shell-escaped like all other template variables.
@wesm
Copy link
Collaborator

wesm commented Feb 3, 2026

thanks!

@wesm wesm merged commit 8138635 into roborev-dev:main Feb 3, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants